Conversation
Switch the EAP Duration Breakdown chart from a line chart to a stacked area chart, remove the extra p90 percentile, use high-fidelity time intervals, alias series names to match the regular chart format (e.g. `p50()` instead of `p50(span.duration)`), and add URL-based legend selection persistence via the `unselectedSeries` query parameter. Co-Authored-By: Claude <noreply@anthropic.com>
…nabled Use `request.method` instead of `http.method` in the transaction summary link filter when the `performance-transaction-summary-eap` flag is enabled, so dashboard filters match the EAP-based summary page. Co-Authored-By: Claude <noreply@anthropic.com>
…r supported keys When the EAP flag is enabled, dashboard widget links filter by request.method instead of http.method. The search query builder didn't recognize request.method as a valid key, causing an "Invalid key" error on the transaction summary page.
github-actions
bot
added
the
Scope: Frontend
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
...app/views/performance/transactionSummary/transactionOverview/useWidgetChartVisualization.tsx
Outdated
Show resolved
Hide resolved
...app/views/performance/transactionSummary/transactionOverview/useWidgetChartVisualization.tsx
Outdated
Show resolved
Hide resolved
…ization Add `?? []` fallback to the plottables mapping so it always resolves to `Plottable[]` even if `spanSeriesData` is nullish. While the isPending/isError guard prevents this in practice, this is the right defensive pattern and matches codebase conventions. Co-Authored-By: Claude <noreply@anthropic.com>
...app/views/performance/transactionSummary/transactionOverview/useWidgetChartVisualization.tsx
Show resolved
Hide resolved
The duration breakdown chart plots percentiles of a single duration metric, not components of a total. Per the TimeSeriesWidgetVisualization guidelines: "Only use area charts if you want to plot multiple series and those series represent components of a total. [...] They are not a good choice for plotting a single duration time series." Reverts the area chart change from 58f0bbc. Co-Authored-By: Claude <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
...app/views/performance/transactionSummary/transactionOverview/useWidgetChartVisualization.tsx
Show resolved
Hide resolved
gggritso
approved these changes
Feb 23, 2026
mjq
deleted the
mjq/2026-02-17/convert-transaction-summary-to-use-eap
branch
mchen-sentry
pushed a commit
that referenced
this pull request
Feb 24, 2026
…ion breakdown chart (#108483) Fix two issues with the EAP transaction summary page and improve the duration breakdown chart. ## Method filter mismatch When the `performance-transaction-summary-eap` flag is enabled, dashboard widget links filter by `request.method` (the EAP field name), but the transaction summary search bar didn't recognize it as a valid key. This caused an "Invalid key" error when navigating from dashboards. Now the search bar accepts `request.method` when EAP is enabled, and the dashboard link uses the correct field name based on the flag. ## Duration breakdown chart The EAP version of the duration breakdown chart now matches the regular (non-EAP) version: - removed the extra p90 percentile - uses high-fidelity time intervals - aliases series names to match the standard format (e.g. `p50()` instead of `p50(span.duration)`) - persists legend selection via the `unselectedSeries` URL query parameter. Note that this is now a line chart instead of an area chart. As per our [Choosing The Plot Type](https://sentry.sentry.io/stories/product/views/dashboards/widgets/timeserieswidget/timeserieswidgetvisualization/#choosing-the-plot-type) guide: > Only use area charts if you want to plot multiple series and those series represent components of a total. For example, area charts are a good choice to show time spent, broken down by span operation. They are not a good choice for plotting a single duration time series. Area charts should be stacked! ## Screens Old Transaction Summary: <img width="1223" height="296" alt="Screenshot 2026-02-23 at 10 53 34" src="https://github.com/user-attachments/assets/2608f2e0-b2ac-40fb-a53d-a0a400b46489" /> EAP version: <img width="1225" height="312" alt="Screenshot 2026-02-23 at 10 49 01" src="https://github.com/user-attachments/assets/47d90d51-bb87-40a6-a7ff-4a63b31b987a" /> Closes DAIN-1226 --------- Co-authored-by: Claude <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fix two issues with the EAP transaction summary page and improve the duration breakdown chart.
Method filter mismatch
When the
performance-transaction-summary-eapflag is enabled, dashboard widget links filter byrequest.method(the EAP field name), but the transaction summary search bar didn't recognize it as a valid key. This caused an "Invalid key" error when navigating from dashboards. Now the search bar acceptsrequest.methodwhen EAP is enabled, and the dashboard link uses the correct field name based on the flag.Duration breakdown chart
The EAP version of the duration breakdown chart now matches the regular (non-EAP) version:
p50()instead ofp50(span.duration))unselectedSeriesURL query parameter.Note that this is now a line chart instead of an area chart. As per our Choosing The Plot Type guide:
Screens
Old Transaction Summary:
EAP version:
Closes DAIN-1226